From: Florian Müllner <fmuellner@gnome.org>
Date: Tue, 9 Jul 2013 21:15:02 +0000 (+0200)
Subject: appDisplay: Use hookup_style() to bind app-view-controls spacing
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~1013^2^2~3875
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=cb4e4bb2db8fe556a2ece742cfb364c0f3e7c807;p=gnome-shell.git

appDisplay: Use hookup_style() to bind app-view-controls spacing

With the monkey-patched ClutterBoxLayout, we no longer need this
code to hook up the 'spacing' property to CSS.

https://bugzilla.gnome.org/show_bug.cgi?id=703905
---

diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js
index 144419cec..9132d18b9 100644
--- a/js/ui/appDisplay.js
+++ b/js/ui/appDisplay.js
@@ -358,17 +358,6 @@ const ControlsBoxLayout = Lang.Class({
         let totalSpacing = this.spacing * (childrenCount - 1);
         return [maxMinWidth * childrenCount + totalSpacing,
                 maxNaturalWidth * childrenCount + totalSpacing];
-    },
-
-    vfunc_set_container: function(container) {
-        if(this._styleChangedId) {
-            this._container.disconnect(this._styleChangedId);
-            this._styleChangedId = 0;
-        }
-        if(container != null)
-            this._styleChangedId = container.connect('style-changed', Lang.bind(this,
-                    function() { this.spacing = this._container.get_theme_node().get_length('spacing'); }));
-        this._container = container;
     }
 });
 
@@ -416,8 +405,9 @@ const AppDisplay = new Lang.Class({
         this.actor.add(this._viewStack, { expand: true });
 
         let layout = new ControlsBoxLayout({ homogeneous: true });
-        this._controls = new St.Widget({ style_class: 'app-view-controls' });
-        this._controls.set_layout_manager(layout);
+        this._controls = new St.Widget({ style_class: 'app-view-controls',
+                                         layout_manager: layout });
+        layout.hookup_style(this._controls);
         this.actor.add(new St.Bin({ child: this._controls }));